Skip to content

Comments

Remove sharded_ddp#456

Merged
regisss merged 1 commit intomainfrom
remove_sharded_ddp
Oct 9, 2023
Merged

Remove sharded_ddp#456
regisss merged 1 commit intomainfrom
remove_sharded_ddp

Conversation

@regisss
Copy link
Collaborator

@regisss regisss commented Oct 9, 2023

What does this PR do?

sharded_ddp is not used in Transformers anymore (see huggingface/transformers#25702) and was never used on Gaudi anyway (since it's linked to Fairscale). Removing all occurrences.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Oct 9, 2023

The documentation is not available anymore as the PR was closed or merged.

@regisss regisss merged commit ca8ecde into main Oct 9, 2023
@regisss regisss deleted the remove_sharded_ddp branch October 9, 2023 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants